Является ли @ $ array ['possible_missing_key'] антипаттерном?

Оператор @ подавляет сообщения об ошибках, и его использование потенциально настраивает ваш код для других ошибок и неожиданного поведения, которое в конечном итоге трудно отследить. Таким образом, это определенно антипаттерн.
Таким образом, я бы предпочел второй бит. Это делает его намного яснее
что его может не быть в массиве, и
какое значение по умолчанию, если его нет
Чтобы сделать его более кратким, вы можете использовать тернарный условный оператор?:, Как показано в ответе Марка Бейкера. Чуть меньше кода и больше символов, но смысл хорошо известен.
|
На самом деле вариант isset - это антипаттерн. Если вы просто используете isset ($ var)? $ Var: NULL с намерением подавить «ошибку», то вы ничего не добьетесь, используя правильный синтаксис для подавления ошибок. Результат такой же, но менее читаемый.
Люди утверждают, что это из-за кажущейся «чистоты» и из-за того, что использование isset - это микрооптимизация. Избегать @ и использовать isset в качестве синтаксической замены соли - это просто культовое программирование.
|
Или же
$ value = (isset ($ array ['possible_missing_key']))? $ array ['возможно_missing_key']: нуль;
|
Игнорирование предупреждений - определенно антипаттерн; так что да, это антипаттерн (и я могу гарантировать, что если вы научитесь подавлять предупреждения, одно из них вернется и укусит вас сзади, если не хуже).
Кроме того, хотя вторая версия более подробна, она дает неинициализированной переменной известное состояние (или может использоваться для решения проблемы, если переменная должна быть заполнена).
|
Третий вариант:
$ value = (isset ($ array ['key'])? $ array ['key']: null);
Я знаю, что это не дает прямого ответа на вопрос; Я бы поставил это как комментарий, но форматировать действительно нужно.
Идея здесь в том, что если вы пытаетесь сделать свой код короче, используя однострочник вместо блока if-else, вы все равно можете превратить его в сжатый однострочник с помощью тернарного оператора, что даст вам лучшее обоих миров.
|
Второй блок кода (или альтернатива Марка Бейкера, которая будет работать точно так же) лучше. Я не совсем уверен насчет PHP, но во многих других языках программирования просто игнорирование переменной почти наверняка вызовет ошибку. По крайней мере, со вторым блоком вы инициализируете переменную каким-то значением или ячейкой памяти.
Подавление ошибок следует чаще использовать, если вы ожидаете, что функция выдаст ожидаемую ошибку в конечном продукте (однако в большинстве случаев это не так).
Удачи!
Деннис М.
|
Ваш ответ
StackExchange.ifUsing ("редактор", function () {
StackExchange.using ("externalEditor", function () {
StackExchange.using ("сниппеты", function () {
StackExchange.snippets.init ();
});
});
}, "фрагменты кода");
StackExchange.ready (функция () {
var channelOptions = {
теги: "" .split (""),
id: "1"
};
initTagRenderer ("". split (""), "" .split (""), channelOptions);
StackExchange.using ("externalEditor", function () {
// Должен запускать редактор после сниппетов, если сниппеты включены
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using ("сниппеты", function () {
createEditor ();
});
}
else {
createEditor ();
}
});
function createEditor () {
StackExchange.prepareEditor ({
useStacksEditor: ложь,
heartbeatType: 'ответ',
autoActivateHeartbeat: ложь,
convertImagesToLinks: правда,
noModals: правда,
showLowRepImageUploadWarning: true,
ReputationToPostImages: 10,
bindNavPrevention: правда,
постфикс: "",
imageUploader: {
brandingHtml: "На основе \ u003ca href = \" https: //imgur.com/ \ "\ u003e \ u003csvg class = \" svg-icon \ "width = \" 50 \ "height = \" 18 \ "viewBox = \ "0 0 50 18 \" fill = \ "none \" xmlns = \ "http: //www.w3.org/2000/svg \" \ u003e \ u003cpath d = \ "M46.1709 9.17788C46.1709 8.26454 46.2665 7.94324 47.1084 7.58816C47.4091 7.46349 47.7169 7.36433 48.0099 7.26993C48.9099 6.97997 49.672 6.73443 49.672 5.93063C49.672 5.22043 48.9832 4.61182 48.1414 4.61182C47.4335 4.61182 46.725623.40716284,61182C47.4335 4.61182 46.725623.4623.68 43.1481 6.59048V11.9512C43.1481 13.2535 43.6264 13.8962 44.6595 13.8962C45.6924 13.8962 46.1709 13.2535 46.1709 11.9512V9.17788Z \ "/ \ u003e \ u003cpath d = \" M32.492 10.1419954.037.048.037.048.048.034.048.048.0148.0148.0148.0148.0148.48.0148.0148.034.048.048.0 148.048.0 37.048.0 37.048.0 148.048.0 148.048.0 37.048.0 37.048.0 37.048.0 37.0 48.0 48.0 148.48.0 148.49.048.0 37.048.0 37.048.0 37.048.0 148.048.48.0 41.5985 12.6954 41.5985 10.1419V6.59049C41.5985 5.28821 41.1394 4.66232 40.1061 4.66232C39.0732 4.66232 38.5948 5.28821 38.5948 6.59049V9.60062C38.5948 10.8521 38.2696 11.5455 37.0451 11.5454.595 1045835.820 521 35.4954 9.60062V6.59049C35.4954 5.28821 35.0173 4.66232 34.0034 4.66232C32.9703 4.66232 32.492 5.28821 32.492 6.59049V10.1419Z \ "/> \ u003cpath fill-rule = \" evenodd \ "clip-rule = \" evenodd \ " = \ "M25.6622 17.6335C27.8049 17.6335 29.3739 16.9402 30.2537 15.6379C30.8468 14.7755 30.9615 13.5579 30.9615 11.9512V6.59049C30.9615 5.28821 30.4833 4.66231 29.4502 4.66233C28.4833 4.66231 29.4502 4.66233C28.41087C 4.58.4833 4.6623.74.9502 4.66231C28.40334.6554.4502 4.66231C28.40337454.659904.58.40334.65 .1369 4.56087 21.01346,57349 21,0134 9.27932C21.0134 11,9852 23,003 13,913 25,3754 13.913C26.5612 13,913 27,4607 13,4902 28,1109 12.6616C28.1109 12,7229 28,1161 12,7799 28,121 12.8346C28.1256 12,8854 28,1301 12,9342 28,1301 12.983C28.1301 14,4373 27,2502 15,2321 25,777 15.2321C24.8349 15,2321 24,1352 14,9821 23,5661 14.7787C23.176 14.6393 22.8472 14.5218 22.5437 14.5218C21.7977 14.5218 21.2429 15.0123 21.2429 15.6887C21.2429 16.7375 22.9072 17.6335 25.6622 17.6335ZM24.1317 9.27932C24.1317 928732.097 26.94329 268 9.27932C24.1317 7.9432.097 24.0167 9284329 26.0117.924.097 24.0167 9.27932C24.1317 7.924.097 24.09 11.5116 26.1024 11.5116C24.9737 11.5116 24.1317 10.6491 24.1317 9.27932Z \ "/ \ u003e \ u003cpath d = \" M16.8045 11.9512C16.8045 13.2535 17.2637 13.8962 18.2965 13.8962C19.3298 13.89351219.80.405 13.8962C19.3298 13.89351219.804.98 4.62866 16.4027 4.62866C15.1594 4.62866 14.279 4.98375 13.3609 5.88013C12.653 5.05154 11.6581 4.62866 10.3573 4.62866C9.34336 4.62866 8.57809 4.89931 7.9466 5.5079C7.58314 4. 9328 7.10506 4.66232 6.51203 4.66232C5.47873 4.66232 5.00066 5.28821 5.00066 6.59049V11.9512C5.00066 13.2535 5.47873 13.8962 6.51203 13.8962C7.54479 13.8962 8.0232 13.2535 8.0232 11.9512V8.9088108.4711.511.511.5118.9411.9511.511.511.9511.511.9511.9511.8448.98.9116.9118.98 .9512C10.893 13.2535 11.3711 13.8962 12.4044 13.8962C13.4375 13.8962 13.9157 13.2535 13.9157 11.9512V8.90741C13.9157 7.58817 14.3365 6.91179 15.4269 6.91179C16.4027 6.91179 16.8045 / 7.58817003168045 .31675 6.59049C3.31675 5.28821 2.83866 4.66232 1.82471 4.66232C0.791758 4.66232 0.313354 5.28821 0.313354 6.59049V11.9512C0.313354 13.2535 0.791758 13.8962 1.82471 13.8962C2.85798 13.8962 3.31675 \ u0033512c \ ud3 \ u00335 \ 3.31675 \ u0033516 "M1.87209 0.400291C0.843612 0.400291 0 1.1159 0 1.98861C0 2.87869 0.822846 3.57676 1.87209 3.57676C2.90056 3.57676 3.7234 2.87869 3.7234 1.98861C3.7234 1.1159 2.90056 0.400291 1.87209 \ B \ B \" u "1.87209 \" \ B \ " \ u003c / svg \ u003e \ u003c / a> ",
contentPolicyHtml: "Вклады пользователей под лицензией \ u003ca href = \" https: //stackoverflow.com/help/licensing \ "\ u003ecc by-sa \ u003c / a \ u003e \ u003ca href = \" https://stackoverflow.com / legal / content-policy \ "> (политика содержания) \ u003c / a>",
allowUrls: true
},
onDemand: правда,
discardSelector: ".discard-answer"
, немедленноShowMarkdownHelp: true, enableTables: true, enableSnippets: true
});
}
});
Спасибо за ответ на Stack Overflow!
Обязательно ответьте на вопрос. Сообщите подробности и поделитесь своим исследованием!
Но избегайте…
Просить о помощи, разъяснениях или отвечать на другие ответы.
Делать заявления, основанные на мнении; подкрепите их рекомендациями или личным опытом.
Чтобы узнать больше, ознакомьтесь с нашими советами по написанию отличных ответов.
Черновик сохранен
Черновик отклонен
Зарегистрируйтесь или войдите под своим ником
StackExchange.ready (функция () {
StackExchange.helpers.onClickDraftSave ('# ссылка для входа');
});
Зарегистрируйтесь с помощью Google
Зарегистрируйтесь через Facebook
Зарегистрируйтесь, используя электронную почту и пароль
Разместить
Опубликовать как гость
имя
Эл. адрес
Обязательно, но не отображается
StackExchange.ready (
function () {
StackExchange.openid.initPostLogin ('. New-post-login', 'https% 3a% 2f% 2fstackoverflow.com% 2fquestions% 2f4441528% 2fis-arraypossably-missing-key-an-anti-pattern% 23new-answer', ' question_page ');
}
);
Опубликовать как гость
имя
Эл. адрес
Обязательно, но не отображается
Разместите свой ответ
Отменить
Нажимая «Опубликовать ответ», вы соглашаетесь с нашими условиями обслуживания, политикой конфиденциальности и политикой использования файлов cookie.
Не тот ответ, который вы ищете? Просмотрите другие вопросы с метками php массивы операторы подавление ошибок или задайте свой вопрос.